Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add HW Runner integration testing CI gate #234

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add HW Runner integration testing CI gate #234

wants to merge 3 commits into from

Conversation

tullom
Copy link
Contributor

@tullom tullom commented Dec 5, 2024 •
edited
Loading

The large amount of changed files is due to all our examples being retrofitted to run integration tests on them.

madeleyneVaca
madeleyneVaca requested changes Dec 5, 2024
View reviewed changes
Copy link
Contributor

@madeleyneVaca madeleyneVaca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the need for the yml file and the test folder, double check that you meant to make so many changes in the examples

@felipebalbi felipebalbi added enhancement New feature or request github_actions Pull requests that update GitHub Actions code labels Dec 11, 2024
jerrysxie
jerrysxie requested changes Jan 3, 2025
View reviewed changes
@tullom tullom force-pushed the workflow branch 2 times, most recently from c0d7213 to 10ddee0 Compare January 9, 2025 00:35
jerrysxie
jerrysxie requested changes Jan 9, 2025
View reviewed changes
@tullom tullom force-pushed the workflow branch 2 times, most recently from 693cba2 to 860fcc5 Compare January 23, 2025 01:22
@tullom
Copy link
Contributor Author

tullom commented Jan 23, 2025 •
edited
Loading

Feel free to try this on your own machine, simply have your board plugged in, fire up Git Bash, $ cd examples/rt685s-evk/src/bin and run $ ./test.sh. It will build, flash, and run all examples in test mode. The sha256-async test is failing in release mode...

@akshat2112 akshat2112 self-requested a review January 31, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jerrysxie jerrysxie jerrysxie approved these changes

@felipebalbi felipebalbi Awaiting requested review from felipebalbi felipebalbi is a code owner

@JamesHuard JamesHuard Awaiting requested review from JamesHuard JamesHuard is a code owner

@madeleyneVaca madeleyneVaca Awaiting requested review from madeleyneVaca madeleyneVaca is a code owner

@akshat2112 akshat2112 Awaiting requested review from akshat2112

Requested changes must be addressed to merge this pull request.

Assignees

@tullom tullom

Labels
enhancement New feature or request github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tullom @felipebalbi @jerrysxie @madeleyneVaca